Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DOC: fix minor documentation errors #426

Merged
merged 1 commit into from
Jan 23, 2024
Merged

Conversation

mferrera
Copy link
Collaborator

Some small things I noticed whilst reading the documentation.

  • Minor grammar/spelling fixes
  • Markdown bullet points that rst didn't translate
  • yml identation
  • sphinx warning

@mferrera mferrera self-assigned this Jan 23, 2024
@@ -380,7 +380,7 @@ class ExportData:

Args:
access_ssdl: Optional. A dictionary that will overwrite or append
to the default ssdl settings read from the config. Example:
to the default ssdl settings read from the config. Example:
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

src/fmu/dataio/dataio.py:docstring of dataio.dataio.ExportData:63:Definition list ends without a blank line; unexpected unindent.

Copy link
Member

@perolavsvendsen perolavsvendsen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! 👍

@mferrera mferrera merged commit 3f1e507 into equinor:main Jan 23, 2024
12 checks passed
@mferrera mferrera deleted the small-doc-fixes branch January 23, 2024 12:53
@perolavsvendsen
Copy link
Member

General comment: The documentation of the schema is manually made, and hence outdated. There has been several attempts to automate it, some have been partially implemented. However, current efforts around Pydantic (@janbjorge) should hopefully give improvements on this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants